Skip to content

remove npm ci timing info#2413

Merged
sawka merged 2 commits intomainfrom
sawka/remove-npm-ci-debugging
Oct 9, 2025
Merged

remove npm ci timing info#2413
sawka merged 2 commits intomainfrom
sawka/remove-npm-ci-debugging

Conversation

@sawka
Copy link
Copy Markdown
Member

@sawka sawka commented Oct 9, 2025

No description provided.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Oct 9, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Two GitHub Actions workflow files were updated. In both .github/workflows/build-helper.yml and .github/workflows/copilot-setup-steps.yml, the npm ci command was modified to remove the --timing and --foreground-scripts flags, retaining only --no-audit and --no-fund. No other steps, jobs, or control flow were changed.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sawka/remove-npm-ci-debugging

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cd2e00e and 72efa82.

📒 Files selected for processing (2)
  • .github/workflows/build-helper.yml (1 hunks)
  • .github/workflows/copilot-setup-steps.yml (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sawka sawka merged commit 47bdc38 into main Oct 9, 2025
3 of 5 checks passed
@sawka sawka deleted the sawka/remove-npm-ci-debugging branch October 9, 2025 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant